chore: replace astro-icon with nimbus-docs Icon, bump nimbus-docs to 0.10.0 - #32647
Merged
Conversation
Tests the astro-icon replacement PR in cloudflare/nimbus#76. Uses pkg.pr.new preview build from commit acfac20.
Contributor
Review✅ No issues found in commit Code ReviewThis code review is in beta and may not always be helpful — use your judgment. No code review issues found. ConventionsNo convention issues found. Style Guide ReviewNo style-guide issues found. CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
Contributor
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
mvvmm
marked this pull request as draft
August 10, 2026 20:30
mvvmm
changed the title
mvvmm
force-pushed
the
chore/nimbus-icon-replacement
branch
from
August 10, 2026 20:37
94e2d42 to
ae599f2
Compare
- Replace all astro-icon imports with @cloudflare/nimbus-docs/components/Icon.astro - Collapse AstroIcon alias to Icon (they were the same component) - Update virtual:astro-icon to virtual:nimbus/icons in directory.astro - Remove astro-icon from astro.config.ts and package.json - Point @cloudflare/nimbus-docs at PR #76 preview build
mvvmm
force-pushed
the
chore/nimbus-icon-replacement
branch
from
August 10, 2026 20:44
ae599f2 to
7850a5d
Compare
Contributor
mvvmm
marked this pull request as ready for review
August 11, 2026 19:09
mvvmm
requested review from
a team,
Refaerds,
celso,
ethulia,
jonnyparris,
kathayl,
kflansburg,
kodster28,
mchenco,
meddulla,
omarmosid,
ruifigueira and
superhighfives
as code owners
August 11, 2026 19:09
mvvmm
changed the title
kodster28
approved these changes
Aug 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Replaces
astro-iconwith the built-in icon system from cloudflare/nimbus#76, and bumps@cloudflare/nimbus-docsto the published0.10.0release.Changes
astro-iconfrompackage.json,astro.config.ts, and all 53 component/page filesimport { Icon } from "astro-icon/components"→import Icon from "@cloudflare/nimbus-docs/components/Icon.astro"AstroIconalias —IconandAstroIconwere the same component; merged all usage toIconvirtual:astro-icon→virtual:nimbus/iconsindirectory.astro@ts-expect-error— the nimbus integration injects types forvirtual:nimbus/icons@cloudflare/nimbus-docsto published0.10.0(waspkg.pr.newpreview pin)astroto^7.2.0to satisfy the0.10.0peer range (>=7.0.0 <7.1.0 || >=7.2.0 <8.0.0)Why
astro-iconstamps alastModifiedtimestamp into its virtual module on every build, invalidating Astro's incremental build cache for thousands of pages. The package hasn't had a commit in 2+ years. Nimbus 0.10.0 replaces it with a built-in Vite plugin that omitslastModified, making the cache stable.The nimbus release also includes a workaround for the css-tree
createRequirebug (svgo#2149) that breaks Vite prerendering — it redirectscss-treeandcssoto their browser bundles which have JSON data inlined.Astro 7.1.x was excluded from Nimbus 0.10.0's peer range due to an upstream static build regression; 7.2.0 fixes that regression.
Verification
pnpm run check— 0 errors, 0 warnings, 0 hintspnpm run lint— cleanpnpm run format:core:check— clean